Skip to content

feat: add public v1 endpoints (CM-967)#3864

Merged
joanagmaia merged 17 commits intomainfrom
feat/CM-967
Mar 6, 2026
Merged

feat: add public v1 endpoints (CM-967)#3864
joanagmaia merged 17 commits intomainfrom
feat/CM-967

Conversation

@skwowet
Copy link
Member

@skwowet skwowet commented Feb 20, 2026

Note

Medium Risk
Introduces new public endpoints and touches member identity/work-experience mutation plus a large refactor of unmerge/data-access behavior; errors could impact member data integrity, cache correctness, or unmerge workflows if edge cases were missed.

Overview
Adds a new OAuth2-protected Public API v1 surface for member and organization operations: resolving members by identities, listing/verifying member identities, reading maintainer roles, CRUD/verify work experiences, and getting/creating organizations, with request validation via zod and a shared validateOrThrow helper.

Refactors member unmerge and cache invalidation: moves unmerge preview/execution + Temporal kickoff into @crowd/common_services, replaces per-service cache invalidation with invalidateMemberQueryCache, and updates internal merge/unmerge handlers and related workers accordingly.

Improves public API observability by logging unhandled errors and sending Slack alerts, tightens request typing via global Express Request augmentation, renames/expands public OAuth scopes, and consolidates affiliation-override DAL functions under member-organization-affiliation while updating member/org DAL methods to support new verified fields and return updated rows.

Written by Cursor Bugbot for commit 8bca20d. This will update automatically on new commits. Configure here.

@skwowet skwowet self-assigned this Feb 20, 2026
Co-authored-by: Cursor <cursoragent@cursor.com>
@skwowet skwowet marked this pull request as ready for review March 4, 2026 05:47
@joanagmaia joanagmaia requested a review from themarolt March 4, 2026 10:30
Copy link
Contributor

@joanagmaia joanagmaia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@skwowet only 1 endpoint missing at this point which is to get the project affiliations with the associated maintainers roles services/apps/data_sink_worker/src/service/member.service.ts.

We can add this after, non-blocking for the PR

@joanagmaia joanagmaia requested a review from themarolt March 4, 2026 15:32
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Copy link
Contributor

@themarolt themarolt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some conflicts to resolve but LGTM.

@joanagmaia joanagmaia merged commit 24de661 into main Mar 6, 2026
16 checks passed
@joanagmaia joanagmaia deleted the feat/CM-967 branch March 6, 2026 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants